Skip to content

feat(docs): Add workflow to automate docs and license generation#38

Merged
OliverTrautvetter merged 47 commits intomainfrom
automate_docs_and_license
Oct 23, 2025
Merged

feat(docs): Add workflow to automate docs and license generation#38
OliverTrautvetter merged 47 commits intomainfrom
automate_docs_and_license

Conversation

@OliverTrautvetter
Copy link
Member

@OliverTrautvetter OliverTrautvetter commented Oct 14, 2025

feat: Automate documentation and license updates

Adds a GitHub Workflow to automatically update all relevant documentation and license information within OMS upon Pull Request.


ClickUp Card

@github-actions github-actions bot force-pushed the automate_docs_and_license branch 2 times, most recently from 45b9bb8 to 23a92d5 Compare October 14, 2025 09:12
Copy link
Member

@NautiluX NautiluX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! I think running the workflow on the PR would actually be good enough. Please see my comments and 🥁 add a description ;)

@OliverTrautvetter OliverTrautvetter force-pushed the automate_docs_and_license branch from e05ce74 to 8884bb1 Compare October 14, 2025 12:31
NautiluX and others added 13 commits October 14, 2025 14:39
Co-authored-by: Simon Herrmann <simonh@codesphere.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
* Adds a description to the service command
* Updates README

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: GitHub Action <action@github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
@OliverTrautvetter OliverTrautvetter force-pushed the automate_docs_and_license branch from 78346c0 to 4756e49 Compare October 14, 2025 12:40
Copy link
Member

@NautiluX NautiluX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

couple comments on the bash script, it looks somehow overly complicated

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Copy link
Member

@NautiluX NautiluX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for updating the binary name! I'm confused by the checkout HEAD logic in the action, would you mind leaving a comment to explain why we need it?

@OliverTrautvetter
Copy link
Member Author

thanks for updating the binary name! I'm confused by the checkout HEAD logic in the action, would you mind leaving a comment to explain why we need it?

ref: ${{ github.event.pull_request.head.ref || github.ref }}

github.event.pull_request.head.ref = if it is a pull request, so Pull Request Branch
github.ref_name = if push to Main

Without this it would always try to push to the standard, so main.

I took the logic from the monorepo where we use it a few times for such things.

Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
Signed-off-by: OliverTrautvetter <66372584+OliverTrautvetter@users.noreply.github.com>
@NautiluX
Copy link
Member

thanks for updating the binary name! I'm confused by the checkout HEAD logic in the action, would you mind leaving a comment to explain why we need it?

ref: ${{ github.event.pull_request.head.ref || github.ref }}

github.event.pull_request.head.ref = if it is a pull request, so Pull Request Branch github.ref_name = if push to Main

Without this it would always try to push to the standard, so main.

I took the logic from the monorepo where we use it a few times for such things.

This is on the checkout action, and as far as I understand the action runs only on pull requests, so github.event.pull_request.head.ref should always be set, right?

Copy link
Member

@NautiluX NautiluX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approving, left 2 smaller comments, the binary name should be updated, the cd repo_root thing doesn't hurt I guess, but I don't get why we need it.

@OliverTrautvetter OliverTrautvetter merged commit 61d9f87 into main Oct 23, 2025
@OliverTrautvetter OliverTrautvetter deleted the automate_docs_and_license branch October 23, 2025 11:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants